Skip to content

Conversation

@Teagan42
Copy link
Contributor

What

  • remove duplicate CrossEncoder import
  • lazily create reranker via PlexServer.reranker
  • search media uses lazy reranker property
  • document lazy reranker design note

Why

  • reduce module-level side effects and avoid unnecessary model downloads

Affects

  • mcp_plex/server.py
  • tests/test_server.py
  • documentation/metadata

Testing

  • uv run ruff check .
  • uv run pytest

Documentation

  • updated AGENTS.md

https://chatgpt.com/codex/tasks/task_e_68c64c68af9c8328a2c27c711f29514b

@github-actions
Copy link

github-actions bot commented Sep 14, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
mcp_plex
   cache.py36489%22, 50–52
   loader.py1971095%32–34, 331, 334–338, 663
   server.py2982093%101–102, 225, 453, 457, 479–485, 517, 527, 558–559, 576–577, 600, 603–604, 663
TOTAL6353495% 

Tests Skipped Failures Errors Time
29 0 💤 0 ❌ 0 🔥 30.901s ⏱️

@Teagan42 Teagan42 merged commit 634a8bc into main Sep 14, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/remove-duplicate-try/except-blocks-in-server.py branch September 14, 2025 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants